Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

p2p: move ping handling into pingLoop goroutine #27887

Merged
merged 1 commit into from
Aug 9, 2023

Conversation

fjl
Copy link
Contributor

@fjl fjl commented Aug 9, 2023

Moving the response sending there allows tracking all peer goroutines in the peer WaitGroup.

Moving the response sending there allows tracking all peer goroutines
in the peer WaitGroup.
@fjl fjl requested a review from zsfelfoldi as a code owner August 9, 2023 11:13
Copy link
Contributor

@holiman holiman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@holiman holiman added this to the 1.12.1 milestone Aug 9, 2023
@fjl fjl merged commit 7ec60d5 into ethereum:master Aug 9, 2023
DarianShawn pushed a commit to dogechain-lab/dbsc that referenced this pull request Sep 3, 2023
### Description

upstream PR:
[go-ethereum#27887](ethereum/go-ethereum#27887)

Moving the response sending there allows tracking all peer goroutines in
the peer WaitGroup.
yperbasis pushed a commit to erigontech/erigon that referenced this pull request Sep 6, 2023
yperbasis pushed a commit to erigontech/erigon that referenced this pull request Sep 6, 2023
iczc pushed a commit to scroll-tech/go-ethereum that referenced this pull request Sep 12, 2023
Moving the response sending there allows tracking all peer goroutines
in the peer WaitGroup.
Thegaram pushed a commit to scroll-tech/go-ethereum that referenced this pull request Sep 12, 2023
* p2p: move ping handling into pingLoop goroutine (ethereum#27887)

Moving the response sending there allows tracking all peer goroutines
in the peer WaitGroup.

* bump version

---------

Co-authored-by: Felix Lange <[email protected]>
@wgr523 wgr523 mentioned this pull request Oct 27, 2023
15 tasks
wgr523 pushed a commit to XinFinOrg/XDPoSChain that referenced this pull request Oct 27, 2023
Moving the response sending there allows tracking all peer goroutines
in the peer WaitGroup.
wgr523 pushed a commit to XinFinOrg/XDPoSChain that referenced this pull request Oct 31, 2023
Moving the response sending there allows tracking all peer goroutines
in the peer WaitGroup.
devopsbo3 pushed a commit to HorizenOfficial/go-ethereum that referenced this pull request Nov 10, 2023
Moving the response sending there allows tracking all peer goroutines
in the peer WaitGroup.
devopsbo3 added a commit to HorizenOfficial/go-ethereum that referenced this pull request Nov 10, 2023
devopsbo3 added a commit to HorizenOfficial/go-ethereum that referenced this pull request Nov 10, 2023
cp-wjhan pushed a commit to cp-yoonjin/go-wemix that referenced this pull request Jan 29, 2024
Moving the response sending there allows tracking all peer goroutines
in the peer WaitGroup.
otherview pushed a commit to vechain/go-ethereum that referenced this pull request Mar 8, 2024
Moving the response sending there allows tracking all peer goroutines
in the peer WaitGroup.
libotony pushed a commit to vechain/go-ethereum that referenced this pull request Mar 8, 2024
Moving the response sending there allows tracking all peer goroutines
in the peer WaitGroup.
jyap808 pushed a commit to ubiq/go-ubiq that referenced this pull request Mar 21, 2024
Moving the response sending there allows tracking all peer goroutines
in the peer WaitGroup.

(cherry picked from commit 7ec60d5)
InnerPeace080 pushed a commit to titantkx/go-ethereum that referenced this pull request Apr 29, 2024
Moving the response sending there allows tracking all peer goroutines
in the peer WaitGroup.
cp-wjhan pushed a commit to cp-wjhan/go-metadium that referenced this pull request May 28, 2024
Moving the response sending there allows tracking all peer goroutines
in the peer WaitGroup.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants